[XEN][POWERPC] cleanup for (;;);

When the port first started, we didn't even have panic() working
properly so we used "for(;;);" loops to stop execution.  Now that output
is buffered these look like hangs, since you usually do not see the
printk() before.

Signed-off-by: Jimi Xenidis <jimix@watson.ibm.com>
Signed-off-by: Hollis Blanchard <hollisb@us.ibm.com>

diff --git a/xen/arch/powerpc/dart_u3.c b/xen/arch/powerpc/dart_u3.c
index d1b38665b396f04bac9d1896d9e43fcf0a7eced4..3058b5763984a316aeec86e4a9f30213d7106140 100644 (file)
--- a/xen/arch/powerpc/dart_u3.c
+++ b/xen/arch/powerpc/dart_u3.c
@@ -55,10 +55,10 @@ static void u3_inv_all(void)
         dc.reg.dc_invtlb = 1;
         out_32(dart_ctl_reg, dc.dc_word);
 
-    do {
-        dc.dc_word = in_32(dart_ctl_reg);
-        r++;
-    } while ((dc.reg.dc_invtlb == 1) && (r < (1 << l)));
+        do {
+            dc.dc_word = in_32(dart_ctl_reg);
+            r++;
+        } while ((dc.reg.dc_invtlb == 1) && (r < (1 << l)));
 
         if (r == (1 << l)) {
             if (l < 4) {

diff --git a/xen/arch/powerpc/external.c b/xen/arch/powerpc/external.c
index 8d60766b8dc70e470a83092f53400b5ee5a32afb..8b9e17a886fab73eb4d8b7189af40fe93883eccd 100644 (file)
--- a/xen/arch/powerpc/external.c
+++ b/xen/arch/powerpc/external.c
@@ -175,8 +175,7 @@ void init_IRQ(void)
 
 void ack_APIC_irq(void)
 {
-    printk("%s: EOI the whole MPIC?\n", __func__);
-    for (;;);
+    panic("%s: EOI the whole MPIC?\n", __func__);
 }
 
 void ack_bad_irq(unsigned int irq)

diff --git a/xen/arch/powerpc/papr/tce.c b/xen/arch/powerpc/papr/tce.c
index 2da90ba7c933a42f96e467db49ccec3fe3cfb209..69efa6517e78bd8e41ad29c66e6880921261f0e5 100644 (file)
--- a/xen/arch/powerpc/papr/tce.c
+++ b/xen/arch/powerpc/papr/tce.c
@@ -47,7 +47,7 @@ static void h_put_tce(struct cpu_user_regs *regs)
         regs->gprs[3] = H_Success;
     }
 }
-    
+
 static void h_get_tce(struct cpu_user_regs *regs)
 {
     u32 liobn = regs->gprs[4];
@@ -57,7 +57,7 @@ static void h_get_tce(struct cpu_user_regs *regs)
     printk("%s: liobn: 0x%x ioba: 0x%lx \n", __func__, liobn, ioba);
 #endif
     regs->gprs[3] = H_Function;
-    for(;;) ;
+    BUG();
 }
 
 static void h_stuff_tce(struct cpu_user_regs *regs)
@@ -76,7 +76,7 @@ static void h_stuff_tce(struct cpu_user_regs *regs)
             count);
 #endif
     regs->gprs[3] = H_Function;
-    for(;;);
+    BUG();
 }
    
 __init_papr_hcall(H_PUT_TCE, h_put_tce);